Skip to content

DAOS-19255 control: Translate net.ErrClosed to Fault (#18572)#18596

Open
kjacque wants to merge 1 commit into
release/2.8from
kjacque/2.8/net-errclosed
Open

DAOS-19255 control: Translate net.ErrClosed to Fault (#18572)#18596
kjacque wants to merge 1 commit into
release/2.8from
kjacque/2.8/net-errclosed

Conversation

@kjacque

@kjacque kjacque commented Jul 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

net.ErrClosed got past our interceptors and was being unhelpfully returned to the user.

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

@kjacque
DAOS-19255 control: Translate net.ErrClosed to Fault (#18572)
67c3c56 
net.ErrClosed got past our interceptors and was being unhelpfully
returned to the user.

Signed-off-by: Kris Jacque <kris.jacque@hpe.com>
@kjacque kjacque self-assigned this Jul 1, 2026
@kjacque kjacque requested review from a team as code owners July 1, 2026 16:26
@kjacque kjacque added approved-to-merge PR has received release branch merge approval release-2.8 clean-cherry-pick Cherry-pick from another branch that did not require additional edits labels Jul 1, 2026
@daltonbohning daltonbohning added this to the release-2.8 milestone Jul 1, 2026
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

Ticket title is 'error reading server preface... use of closed network connection'
Status is 'Awaiting backport'
Labels: 'test_2.8.0rc'
Job should run at elevated priority (1)
https://daosio.atlassian.net/browse/DAOS-19255

@github-actions github-actions Bot added the priority Ticket has high priority (automatically managed) label Jul 1, 2026
@kjacque kjacque added the forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. label Jul 6, 2026
@kjacque

kjacque commented Jul 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Looks like all testing completed with no errors in Jenkins, looks like the unit test with bdev stage was disabled in the parent branch.

@kjacque kjacque requested a review from a team July 6, 2026 14:48
@kjacque

kjacque commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

@daos-stack/daos-gatekeeper ping...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tanabarr tanabarr tanabarr approved these changes
@daltonbohning daltonbohning daltonbohning approved these changes
@mjmac mjmac Awaiting requested review from mjmac

Assignees

@kjacque kjacque

Labels

approved-to-merge PR has received release branch merge approval clean-cherry-pick Cherry-pick from another branch that did not require additional edits forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. priority Ticket has high priority (automatically managed) release-2.8

Milestone

release-2.8

Development

Successfully merging this pull request may close these issues.

3 participants

@kjacque @tanabarr @daltonbohning